Skip to content

About notifications - #3371

Merged
adriendupuis merged 34 commits into
5.0from
enh-notifications
Sep 9, 2026
Merged

About notifications#3371
adriendupuis merged 34 commits into
5.0from
enh-notifications

Conversation

@adriendupuis

@adriendupuis adriendupuis commented Aug 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
Question Answer
JIRA Ticket
Versions
Edition

Inspired by ibexa/recipes-dev#257

  • Details when and to whom the notification are sent.
  • Info about which notification type a channel accept is needed even for built-in type subscriptions, so I moved/merged the table up.
  • Link to this table.
  • I showed default subscriptions for admin and site, I did it with 257 applied.
  • I made few edits in other pages I'm targetting.
    • Details a bit more participant types and scopes, opportunity to add PHP API Ref links.
    • Add /from-invite/register to the self registering user permission case.
    • format permission the way (downcase and slash) that seem usual in the rest of the doc.

Checklist

  • Text renders correctly
  • Text has been checked with vale
  • Description metadata is up to date
  • Redirects cover removed/moved pages
  • Code samples are working
  • PHP code samples have been fixed with PHP CS fixer
  • Added link to this PR in relevant JIRA ticket or code PR

@github-actions

github-actions Bot commented Aug 28, 2026 •
edited
Loading

Copy link
Copy Markdown

Preview of modified files

Preview of modified Markdown:

@github-actions

github-actions Bot commented Aug 28, 2026 •
edited
Loading

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 750785
🔗 Unique 15085
✅ Successful 6320
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 744465
❓ Unknown 0
🚫 Errors 0
⛔ Unsupported 0

Full Github Actions output

@adriendupuis
adriendupuis marked this pull request as ready for review August 28, 2026 08:29
mnocon
mnocon reviewed Aug 28, 2026
View reviewed changes
Comment thread docs/api/notification_channels.md Outdated
adriendupuis
adriendupuis commented Aug 28, 2026
View reviewed changes
Comment thread docs/api/notification_channels.md
@adriendupuis
adriendupuis marked this pull request as draft August 31, 2026 07:55
@adriendupuis adriendupuis changed the title About admin notifications About notifications Sep 1, 2026
@adriendupuis adriendupuis mentioned this pull request Sep 1, 2026
Merged
5 tasks
adriendupuis
adriendupuis commented Sep 1, 2026
View reviewed changes
Comment thread docs/users/invitations.md Outdated
adriendupuis
adriendupuis commented Sep 2, 2026
View reviewed changes
Comment thread docs/content_management/collaborative_editing/collaborative_editing_api.md Outdated
@adriendupuis
adriendupuis marked this pull request as ready for review September 2, 2026 14:50
@adriendupuis
adriendupuis requested a review from mnocon September 2, 2026 14:50
mnocon
mnocon reviewed Sep 7, 2026
View reviewed changes
Comment thread docs/permissions/permission_use_cases.md Outdated
Comment thread docs/content_management/collaborative_editing/collaborative_editing_api.md Outdated
Comment thread docs/content_management/collaborative_editing/collaborative_editing_api.md Outdated
Comment thread docs/api/notification_channels.md Outdated
Comment thread docs/api/notification_channels.md Outdated
Comment thread docs/api/notification_channels.md Outdated
Comment thread docs/api/notification_channels.md Outdated
Comment thread docs/api/notification_channels.md Outdated
@adriendupuis
adriendupuis requested a review from mnocon September 8, 2026 10:49
mnocon
mnocon reviewed Sep 9, 2026
View reviewed changes
Comment thread docs/content_management/collaborative_editing/collaborative_editing_api.md Outdated
@adriendupuis @mnocon
Apply suggestion from @mnocon
63ab3af 
Co-authored-by: Marek Nocoń <mnocon@users.noreply.github.com>
@adriendupuis
adriendupuis requested a review from mnocon September 9, 2026 11:56
mnocon
mnocon approved these changes Sep 9, 2026
View reviewed changes

@mnocon mnocon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

code_samples/ change report

Before (on target branch)After (in current PR)

code_samples/api/notifications/notification_send.php


code_samples/api/notifications/notification_send.php

docs/api/notification_channels.md@152:``` php hl_lines="11-14"
docs/api/notification_channels.md@153:[[= include_code('code_samples/api/notifications/notification_send.php', 2) =]]
docs/api/notification_channels.md@154:```
docs/api/notification_channels.md@229:``` php hl_lines="13-16"
docs/api/notification_channels.md@230:[[= include_code('code_samples/api/notifications/notification_send.php', 2) =]]
docs/api/notification_channels.md@231:```

001⫶use App\Notifications\MyNotification; // extends Symfony\Component\Notifier\Notification\Notification
002⫶use Ibexa\Contracts\Notifications\Value\Notification\SymfonyNotificationAdapter;
003⫶use Ibexa\Contracts\Notifications\Value\Recipent\SymfonyRecipientAdapter;
004⫶use Ibexa\Contracts\Notifications\Value\Recipent\UserRecipient;
005⫶
006⫶$subject = 'My subject';
007⫶

001⫶use App\Notifications\MyNotification; // extends Symfony\Component\Notifier\Notification\Notification
002⫶use Ibexa\Contracts\Notifications\Value\Notification\SymfonyNotificationAdapter;
003⫶use Ibexa\Contracts\Notifications\Value\Recipent\SymfonyRecipientAdapter;
004⫶use Ibexa\Contracts\Notifications\Value\Recipent\UserRecipient;
005⫶
006⫶$subject = 'My subject';
007⫶
008⫶/** @var \Ibexa\Contracts\Notifications\Service\NotificationServiceInterface $notificationService */
009⫶/** @var \Ibexa\Contracts\Core\Repository\UserService $userService */
010⫶/** @var \Ibexa\Contracts\Core\Repository\PermissionResolver $permissionResolver */
011❇️$notificationService->send(
012❇️    new SymfonyNotificationAdapter(new MyNotification($subject)),
013❇️    [new SymfonyRecipientAdapter(new UserRecipient($userService->loadUser($permissionResolver->getCurrentUserReference()->getUserId())))],
008⫶/**
009⫶ * @var \Ibexa\Contracts\Notifications\Service\NotificationServiceInterface $notificationService
010⫶ * @var \Ibexa\Contracts\Core\Repository\UserService $userService
011⫶ * @var \Ibexa\Contracts\Core\Repository\PermissionResolver $permissionResolver
012⫶ */
013❇️$notificationService->send(
014❇️    new SymfonyNotificationAdapter(new MyNotification($subject)),
015❇️    [new SymfonyRecipientAdapter(new UserRecipient($userService->loadUser($permissionResolver->getCurrentUserReference()->getUserId())))],


 Download colorized diff

@adriendupuis
adriendupuis merged commit c5f2874 into 5.0 Sep 9, 2026
13 checks passed
@adriendupuis
adriendupuis deleted the enh-notifications branch September 9, 2026 13:25
adriendupuis added a commit that referenced this pull request Sep 9, 2026
@adriendupuis @mnocon
About notifications (#3371)
766f8c4 
* notification_channels.md: About admin notifications
* notification_channels.md: About FormSubmitted
* notification_channels.md: About default recipients
* notification_channels.md: Continue with triggers and senders
* notification_channels.md: Remove SystemNotification
* notification_channels.md: Rework Available notification channels

---------

Co-authored-by: Marek Nocoń <mnocon@users.noreply.github.com>
adriendupuis added a commit that referenced this pull request Sep 9, 2026
@adriendupuis @mnocon
About notifications (#3371)
8e9f769 
* notification_channels.md: About admin notifications
* notification_channels.md: About FormSubmitted
* notification_channels.md: About default recipients
* notification_channels.md: Continue with triggers and senders
* notification_channels.md: Remove SystemNotification
* notification_channels.md: Rework Available notification channels

---------

Co-authored-by: Marek Nocoń <mnocon@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mnocon mnocon mnocon approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@adriendupuis @mnocon